Spark as starter theme, legacy identifiers callout, Theme Kit links, disputes and campaign notes, reference redirect - #49
Merged
Merged
Conversation
…disputes and campaign-order notes, reference redirect - Getting Started and Theme Kit guide: Spark is the starter theme; Intro Bootstrap named as the older Bootstrap-based alternative. - Admin API overview: 'Why 29next?' callout: former name; hostnames, headers, API key namespace carry it; current, not scheduled to change. Same wording in llms.txt and AGENTS.md. - Themes overview: links its own Theme Kit page instead of GitHub (a clean-agent run guessed a wrong URL and concluded the commands were off-site). - Webhooks: dispute events cover chargebacks and alerts. Campaigns: campaign orders are regular store orders. - AGENTS.md corrections: no public route yet, use your support contact. - Redirect /docs/admin-api/reference -> /docs/admin-api (old Redoc URL still linked nine times from the platform changelog, was a 404). Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
developer-docs | 7e3b0c7 | Commit Preview URL Branch Preview URL |
Sep 03 2026, 09:22 AM |
next-devin
marked this pull request as ready for review
September 3, 2026 09:10
![kilo-code-bot[bot]](https://avatars.githubusercontent.com/in/2193792?s=60&v=4){kind=small}
Code Review SummaryStatus: No Issues Found | Recommendation: Merge All 5 issues from the previous review are addressed in 7e3b0c7:
No new issues found on the incremental diff. Files Reviewed (4 changed files)
Previous Review Summary (commit eef8008)Current summary above is authoritative. Previous snapshots are kept for context only. Previous review (commit eef8008)Status: 5 Issues Found | Recommendation: Address before merge Overview
Issue Details (click to expand)WARNING
SUGGESTION
Files Reviewed (10 files)
Reviewed by minimax-m3 · Input: 29K · Output: 2.3K · Cached: 197.7K |
…ute callout wording, redirect comments and trailing-slash form The changelog links the old reference URL with a trailing slash and a query string; Cloudflare exact-path rules do not cover that variant, so both forms are listed. Deep reference pages verified unaffected on the branch preview. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to #48. The wording that waited on the product owner, plus three small things two clean-agent evaluation runs on 2026-09-03 tripped over.
What
NextCommerceCo/spark; Intro Bootstrap is named as the older Bootstrap-based alternative in the Theme Kit warning.X-29next-API-VersionandX-29Next-Signatureheaders, and the API key namespace still carry that name; current on every store, not scheduled to change. Same sentence in/llms.txtandAGENTS.md. One evaluation run found the rename only in llms.txt and a second missed it entirely because its fetch tool summarised the file./docs/themes/theme-kit, got a 404, and reported the CLI commands as undocumented.dispute.*covers chargebacks and pre-chargeback alerts; nothing said "chargeback" before. Campaigns: one sentence that campaign orders are regular store orders.AGENTS.mdcorrections line: no public correction route yet; use your support contact./docs/admin-api/referenceto/docs/admin-apiin bothpublic/_redirectsandnetlify.toml. The old Redoc-style URL is linked nine times from the platform changelog and was a 404.Verified
npm run build,npm run check-agent-surfacespass.validate-linksstill reports only the pre-existing h4-fragment error in the fulfillment-service guide.Decision needed from — none; the starter theme and REST wording were Alex's answers, the support route stays open by his call.
Sibling PR: NextCommerceCo/docs#33.
🤖 Generated with Claude Code